Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Exit JVM when OOM #15538

Merged
merged 1 commit into from
Jan 31, 2024
Merged

Conversation

ruanwenjun
Copy link
Member

@ruanwenjun ruanwenjun commented Jan 31, 2024

Purpose of the pull request

Avoid JVM doesn't exit when OOM occur. This may cause some problem, e.g. the Server cannot work but still in registry and the server crash alert doesn't send.

Brief change log

Add -XX:+ExitOnOutOfMemoryError in jvm_args_env.sh, this should work on jvm version > 8.

Verify this pull request

This pull request is code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(or)

If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

@ruanwenjun ruanwenjun force-pushed the dev_wenjun_ExitOnOutOfMemoryError branch from b901c4d to acd4152 Compare January 31, 2024 03:31
@ruanwenjun ruanwenjun added improvement make more easy to user or prompt friendly 3.2.1 labels Jan 31, 2024
@ruanwenjun ruanwenjun self-assigned this Jan 31, 2024
@codecov-commenter
Copy link

codecov-commenter commented Jan 31, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (3640a28) 38.15% compared to head (179054f) 38.15%.

❗ Current head 179054f differs from pull request most recent head acd4152. Consider uploading reports for the commit acd4152 to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##                dev   #15538   +/-   ##
=========================================
  Coverage     38.15%   38.15%           
- Complexity     4707     4708    +1     
=========================================
  Files          1304     1304           
  Lines         44828    44828           
  Branches       4807     4807           
=========================================
+ Hits          17105    17106    +1     
  Misses        25870    25870           
+ Partials       1853     1852    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

sonarcloud bot commented Jan 31, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@ruanwenjun ruanwenjun merged commit 25810a8 into apache:dev Jan 31, 2024
56 checks passed
@ruanwenjun ruanwenjun deleted the dev_wenjun_ExitOnOutOfMemoryError branch January 31, 2024 05:59
zhongjiajie pushed a commit that referenced this pull request Feb 6, 2024
(cherry picked from commit 25810a8)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.2.1 backend improvement make more easy to user or prompt friendly ready-to-merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants